Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update changes-of-context.html #1765

Merged
merged 3 commits into from
Mar 18, 2022
Merged

Update changes-of-context.html #1765

merged 3 commits into from
Mar 18, 2022

Conversation

mbgower
Copy link
Contributor

@mbgower mbgower commented Apr 23, 2021

removal of "content" limiter from major changes, in an attempt to clarify one aspect of #737 without having an impact on the rest of the standard

Closes #737

removal of "content" limiter from major changes
@patrickhlauke
Copy link
Member

I think/thought that based on @alastc's comment #737 (comment) this PR might still be relevant/could be reopened?

@alastc alastc reopened this Jan 7, 2022
Copy link
Contributor

@bruce-usab bruce-usab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Discussion on call 2/1 got more than a bit sidetracked. See minutes.

removal of "content" limiter from major changes

My understanding is that "from major changes" refers to another PR which AG WG did not reach consensus.

This PR is intended to be editorial only, This PR removes "content of the Web Page" from the beginning of changes-of-context definition because there is a lexical contradiction with the current phrasing.

@patrickhlauke
Copy link
Member

now i am utterly confused...

@alastc
Copy link
Contributor

alastc commented Feb 2, 2022

Discussion is in #737

@bruce-usab
Copy link
Contributor

bruce-usab commented Feb 2, 2022

now i am utterly confused...

Sorry @patrickhlauke as that was the opposite effect I was hoping for,!

On the call yesterday I was thrown by reading this PR (and Mike's terse description) before the discussion thread. Based on the live conversation (and as reflected in the minutes), I was not the only one confused. I thought by commenting here in the PR that I was helping.

added "major" back in
@alastc alastc added Errata Erratum to a Recommendation WCAG 2.1 labels Feb 8, 2022
Copy link

@cliffbert cliffbert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe need to address changes in focus, window, etc. more clearly, although mentioned later in the list below. e.g. "Major changes in content or focus"? "Major changes in the following, ..."?

@patrickhlauke
Copy link
Member

@cliffbert i'm afraid that ship has sailed, after 2+ weeks of back and forth discussions (and see the whole lengthy discussion on the original topic #737 as well, and counter-proposals, and two rounds of voting in AGWG meetings, etc). i'd say this is the least bad correction that everybody could agree on...which is still better than what it was before.

@alastc
Copy link
Contributor

alastc commented Mar 18, 2022

Agreed: https://lists.w3.org/Archives/Public/w3c-wai-gl/2022JanMar/0095.html

NB: NOT deleting the branch until I'm sure it's in the WCAG2.1 branch as well.

@alastc alastc merged commit 8a82316 into main Mar 18, 2022
@fstrr fstrr deleted the change-of-context-mg branch January 13, 2023 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Errata Erratum to a Recommendation WCAG 2.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"changes of context" definition complete?
5 participants